Skip to content
This repository has been archived by the owner on Nov 29, 2017. It is now read-only.

Fixes #3 Add possibility to pass construct argument #11

Open
wants to merge 1 commit into
base: behat3
Choose a base branch
from
Open

Fixes #3 Add possibility to pass construct argument #11

wants to merge 1 commit into from

Conversation

extdevelopment
Copy link

No description provided.

to pass construct argument
@shouze
Copy link
Contributor

shouze commented Sep 9, 2015

Thank you @extdevelopment :)

I haven't checked yet but are you sure than config property can't be used to do that? Actual code lacks a bit of documentation & tests about that so I'll go deeper to check that on my side prior to merge this PR.

I guess you will to have to make the same PR against master branch (behat 2).

@extdevelopment
Copy link
Author

@shouze thx you that you check my PR.

I'm not sure the original goal with config, not documented, possible to use construct argument,
but not in its the present situation.
https://github.com/rezzza/mock-extension/blob/behat3/src/Adapter/AtoumAdapter.php#L42

call_user_func($this->config, $this->mockGenerator);

It would be necessary to define the structure of the config.

If it cleared the right solution, I'm going to do behat 2 master branch also.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants